Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add hpa for tracing #3159

Merged
merged 2 commits into from
Jul 24, 2023
Merged

feat: add hpa for tracing #3159

merged 2 commits into from
Jul 24, 2023

Conversation

sumo-drosiek
Copy link
Contributor

Adds HPA for tracing components we can add it:

  • tracesGateway
  • otelcolInstrumentation

@github-actions github-actions bot added the documentation documentation label Jul 20, 2023
@sumo-drosiek sumo-drosiek marked this pull request as ready for review July 20, 2023 09:28
@sumo-drosiek sumo-drosiek requested a review from a team as a code owner July 20, 2023 09:28
@swiatekm
Copy link

I think the reason we haven't done this yet is that different replica counts require different cascading filter settings. Right @mat-rumian ?

@sumo-drosiek
Copy link
Contributor Author

I think the reason we haven't done this yet is that different replica counts require different cascading filter settings. Right @mat-rumian ?

This is true for trace sampler, which I'm not touching in this PR

Signed-off-by: Dominik Rosiek <[email protected]>
Signed-off-by: Dominik Rosiek <[email protected]>
@sumo-drosiek sumo-drosiek merged commit 847d189 into main Jul 24, 2023
40 checks passed
@sumo-drosiek sumo-drosiek deleted the drosiek-tracing-hpa branch July 24, 2023 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants